Skip to content

Conversation

@molakalapalliharipriya
Copy link
Contributor

No description provided.

shripadbpersonal and others added 5 commits November 19, 2025 15:22
…meter to prevent Deep Sleep playback failures

Reason for change:
Linear and VOD playback fails after devices come out of Deep Sleep state due to stale DRM handles that are not properly released before entering Deep Sleep mode. The current stop() implementation does not provide a mechanism to force DRM handle cleanup, leading to tune failures with stale DRM sessions after device wake-up.

Changes done:
- Enhanced PlayerInstanceAAMP::Stop() API with optional forceCleanup parameter (default false)
- Added DRM session cleanup logic using licenseManager->clearDrmSession(forceClearSession) and clearFailedKeyIds()
- Updated JavaScript bindings to support both single parameter stop(forceCleanup) and two parameter stop(sendStateChangeEvent, forceCleanup) forms
- Modified jsmediaplayer.cpp to handle argument parsing for backward compatibility
- Updated UVE API documentation with usage examples for Deep Sleep scenarios
- Enhanced mock files and test infrastructure to maintain compatibility
- Added comprehensive logging for debugging Deep Sleep DRM cleanup operations

This allows applications to call player.stop(true) before Deep Sleep to proactively clean up DRM resources and prevent playback failures after wake-up.

Test Procedure:

Priority: P2

Risks:Low
@molakalapalliharipriya molakalapalliharipriya requested a review from a team as a code owner November 24, 2025 13:58
@shripadbpersonal shripadbpersonal changed the title Feature/vplay 11809 vipa VPLAY-11809 [VIPA][AAMP] Enhance UveAAMP stop() API with DRM handle cleanup parameter to prevent Deep Sleep playback failures Nov 30, 2025
…k-to-back; Stop() invoked without arguments and as part of Destroy/Release internal call to stop releases all the DRM resources
…k-to-back; Stop() invoked without arguments and as part of Destroy/Release internal call to stop releases all the DRM resources
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants